Skip to content

feat: add env var to disable file change summaries#200

Closed
davdroman wants to merge 1 commit intorynfar:mainfrom
davdroman:feat/disable-file-changes
Closed

feat: add env var to disable file change summaries#200
davdroman wants to merge 1 commit intorynfar:mainfrom
davdroman:feat/disable-file-changes

Conversation

@davdroman
Copy link
Copy Markdown
Contributor

@davdroman davdroman commented Mar 31, 2026

Summary

  • Adds MERIDIAN_NO_FILE_CHANGES env var to disable the "Files changed" summary appended to responses
  • Guards all three injection points: PostToolUse hook registration, non-streaming summary, and streaming SSE block emission

Motivation

The file change summary introduced in #189 is too noisy for some setups and should be opt-in, or at the very least let users opt out. This adds the opt-out mechanism via env var.

Usage

MERIDIAN_NO_FILE_CHANGES=1 npm start

@davdroman davdroman force-pushed the feat/disable-file-changes branch from 7ab6b22 to 1ce61b8 Compare March 31, 2026 08:54
@rynfar
Copy link
Copy Markdown
Owner

rynfar commented Apr 1, 2026

thanks for this submission, i am going to adopt this with one minor change.

@rynfar
Copy link
Copy Markdown
Owner

rynfar commented Apr 1, 2026

Thanks for this — the motivation and implementation were solid. We landed it in #209 with three small adjustments:

CLAUDE_PROXY_NO_FILE_CHANGES fallback added
Every other env var in the codebase uses MERIDIAN_X ?? CLAUDE_PROXY_X so users on the older prefix can still use it. Updated to:

const trackFileChanges = !(process.env.MERIDIAN_NO_FILE_CHANGES ?? process.env.CLAUDE_PROXY_NO_FILE_CHANGES)

4 regression tests added
The project requires tests for all behaviour changes. Added to proxy-file-changes.test.ts: hook suppression, non-streaming summary suppressed, streaming SSE block suppressed, and the CLAUDE_PROXY_ fallback.

README env var table updated
MERIDIAN_NO_FILE_CHANGES | CLAUDE_PROXY_NO_FILE_CHANGES added to the configuration table so users can discover it.

Closing in favour of #209, now merged to main.

@rynfar rynfar closed this Apr 1, 2026
@davdroman davdroman deleted the feat/disable-file-changes branch April 3, 2026 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants